Skip to content

New property to indicate an interchange within the same vehicle#1027

Merged
TuThoThai merged 3 commits into
v2.1-wipfrom
changeWithinVehicle
Jul 16, 2026
Merged

New property to indicate an interchange within the same vehicle#1027
TuThoThai merged 3 commits into
v2.1-wipfrom
changeWithinVehicle

Conversation

@duexw

@duexw duexw commented May 22, 2026

Copy link
Copy Markdown
Contributor

In the case where trains split for different final destinations, this can be modelled as two (or more) ServiceJourneys with a ServiceJourneyInterchange. A passenger may remain seated if he/she entered the correct train part in the first place. Setting "StaySeated" true seems to be somwhat unsafe in this situation, as the passenger may end up in the wrong destination. We therefore propose to introduce a new Element "ChangeWithinVehicle" in InterchangePropertiesGroup. If set to true, the passenger may have to change within the vehicle.

@duexw duexw added the enhancement non semantic enhacement: technical enhancement, etc. label May 22, 2026
@ue71603

ue71603 commented May 23, 2026

Copy link
Copy Markdown
Contributor

I think this PR is useful, because:

  • ServiceJourneyInterchange is the element favored in EPIP.
  • It aligns with what OJP doe.

@ue71603 ue71603 added this to the netex_2.1 milestone May 23, 2026
ue71603
ue71603 previously approved these changes May 23, 2026
skinkie
skinkie previously approved these changes May 23, 2026
trurlurl
trurlurl previously approved these changes May 25, 2026
@ue71603

ue71603 commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

@thbar @TuThoThai can somebody outside of the people that have already discussed this also check this PR and approve it? So that we can merge it?

@ue71603

ue71603 commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

@thbar ready to merge. Or should I do it?

@thbar

thbar commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

@ue71603 I will be able to respond fully next week, but not this one.

Adding the tag `+v2.1` in the documentation of the added element
@TuThoThai TuThoThai dismissed stale reviews from trurlurl, skinkie, and ue71603 via f76c1cb July 16, 2026 15:15
@TuThoThai

Copy link
Copy Markdown
Contributor

This PR was previously approved by @skinkie, @trurlurl and @ue71603
I merge it after I have added my own review that includes a documentation change + addressing the previously failed GitHub Action

@TuThoThai TuThoThai merged commit 194ddc6 into v2.1-wip Jul 16, 2026
2 checks passed
@TuThoThai TuThoThai deleted the changeWithinVehicle branch July 16, 2026 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement non semantic enhacement: technical enhancement, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants